Skip to content

Strictly parse trusted proxy count - #147

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/qryptchat-next-bug
Jul 12, 2026
Merged

Strictly parse trusted proxy count#147
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/qryptchat-next-bug

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • reject malformed TRUSTED_PROXY_COUNT values instead of partially accepting them via parseInt
  • reuse the shared getClientIp implementation from middleware to avoid drift
  • add focused tests for trusted proxy parsing and X-Forwarded-For selection

Validation

  • vitest run src/lib/server/rate-limiter.test.js
  • node --check src/lib/server/rate-limiter.js
  • node --check src/middleware.js
  • git diff --check

@ralyodio
ralyodio merged commit 580d24c into profullstack:master Jul 12, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants